Skip to content

fix(group): pass previous display name in GroupChangedEvent#58990

Open
mosi-kha wants to merge 1 commit intonextcloud:masterfrom
mosi-kha:fix/group-displayname-event-oldvalue
Open

fix(group): pass previous display name in GroupChangedEvent#58990
mosi-kha wants to merge 1 commit intonextcloud:masterfrom
mosi-kha:fix/group-displayname-event-oldvalue

Conversation

@mosi-kha
Copy link

@mosi-kha mosi-kha commented Mar 17, 2026

Summary

GroupChangedEvent old value was empty string,

TODO

  • fix: now passes previous display name,

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Signed-off-by: mostafa <mostafakhaki00@gmail.com>
@mosi-kha mosi-kha requested a review from a team as a code owner March 17, 2026 09:25
@mosi-kha mosi-kha requested review from CarlSchwan, artonge, leftybournes and salmart-dev and removed request for a team March 17, 2026 09:25
@mosi-kha
Copy link
Author

Hi
if possible, backport it into stable32.
thanks

Copy link
Collaborator

@artonge artonge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense.

In which context do you need that information?

@mosi-kha
Copy link
Author

Makes sense.

In which context do you need that information?

Thanks for the approval.
I’m handling group change events and needed access to both the old and new values for additional processing. Since the old value was not included in the event, I introduced a patch to provide it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants